Skip to content

LC Course Prototype#36909

Closed
kdmccormick wants to merge 35 commits intomasterfrom
lc-course-prototype
Closed

LC Course Prototype#36909
kdmccormick wants to merge 35 commits intomasterfrom
lc-course-prototype

Conversation

@kdmccormick
Copy link
Member

@kdmccormick kdmccormick commented Jun 13, 2025

@kdmccormick kdmccormick mentioned this pull request Jun 13, 2025
Closed
@kdmccormick
Copy link
Member Author

kdmccormick commented Jun 13, 2025

WIP:

kdmccormick and others added 15 commits June 17, 2025 16:35
@kdmccormick
fix: Create ContainerVersion subclassses too
2f337d1 
This commit stops this error:

   ContainerVersion.unitversion.RelatedObjectDoesNotExist: ContainerVersion has no unitversion.

from happening when accessing migrated content.

You will have to re-migrate the content to fix. You may even have to discard the
original broken migrated content.
@kdmccormick
Merge pull request #36913 from openedx/lc-course-prototype-migrate-ou…
ef49743 
…tline

feat: Migrate an OutlineRoot into a library
@kdmccormick
fix: drop all the models to un-bork modulestoreblocksource
5b72b4d
@kdmccormick
fix: recreate models with missing db constraint
dc16ae0 
we were missing the unique_together on modulestoreblocksource,
meaning we had multiple modulestoreblocksources for the
same legacy usage key.

also, replace bulk_create with a series of saves to un-bork
forwarding for now.

also, this renames forwarded_by -> forwarded
@Agrendalath @ormsbee
feat: store XBlock field data in MySQL
d5d8118
@Agrendalath @ormsbee
fixup! feat: store XBlock field data in MySQL
4f58e4c
@Agrendalath @ormsbee
fixup! feat: store XBlock field data in MySQL
6ba5c97
@ormsbee
feat: hacky splitmodulestore shim writing from learning core
4eb6343
@ormsbee
temp: more debugging
5cfc04e
@ormsbee
temp: disable shim
8f1ee6b
@ormsbee
temp: writing parent/child relations for import works
5008fbb
@ormsbee
temp: the most basic editing in library works, but you still have to …
789a46f 
…rebuild the structure doc manually
@ormsbee
temp: switch LC vs. ModuleStore based on lookup table
f407a69
@ormsbee
temp: LC course publishing from library
9732f20
@ormsbee
temp: auto-add new content to containers, disable outline caching
2dae58a
@kdmccormick kdmccormick added the create-sandbox A test sandbox will be created for this PR, using the `open-craft/pr-sandbox-automation` tool label Jun 24, 2025
@open-craft-grove
Copy link

open-craft-grove commented Jun 24, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@kdmccormick kdmccormick mentioned this pull request Jun 24, 2025
Closed
@open-craft-grove
Copy link

open-craft-grove commented Jun 25, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Jun 25, 2025

Sandbox deployment failed 💥
Please check the settings and requirements.
Retry deployment by pushing a new commit or updating the requirements/settings in the pull request's description.
📜 Failure Logs
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@open-craft-grove
Copy link

open-craft-grove commented Jun 26, 2025

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@ormsbee
fix: don't add stuff to the structure if we don't have field data for it
f20fc5c 
Also special case ignore "discussion" blocks, for those libraries where
we already wrote that data into the structures in places.
@open-craft-grove
Copy link

open-craft-grove commented Jun 29, 2025

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@bradenmacdonald
Copy link
Contributor

bradenmacdonald commented Jun 30, 2025

For the record, here's how I set up the sandbox for this:

  1. Import the demo course
  2. Create an empty library
  3. Go to https://studio.pr-36909-139931.sandboxes.opencraft.hosting/admin/modulestore_migrator/modulestoresource/add/
    Add source with key course-v1:... from step 1.
  4. Go to https://studio.pr-36909-139931.sandboxes.opencraft.hosting/admin/modulestore_migrator/modulestoresource/
    Action: Start migration (select the course from step 3), Target: library key from step 2 (e.g. lib:OpenedX:Demo), Replace: YES, Aggregate up to CourseRun, Go
  5. Go to the library and publish all.
  6. Repeat steps 4 and 5 (workaround required for issues in the prototype)
  7. Verify at https://studio.pr-36909-139931.sandboxes.opencraft.hosting/admin/xblock_new/learningcorelearningcontext/ that the course is now enabled for Learning Core
  8. Go to https://app.pr-36909-139931.sandboxes.opencraft.hosting/learner-dashboard/ and view the course!

@kdmccormick
Copy link
Member Author

kdmccormick commented Aug 14, 2025

Talk slides are here: https://docs.google.com/presentation/d/1jxl67EC7dyo3NMXf2yk_eZKLLtBEoh7LvtsbXKnxxXs/edit?slide=id.g36961410b1f_0_8#slide=id.g36961410b1f_0_8

Talk video including a demo of this PR in action will be uploaded soon to https://www.youtube.com/@OpenedX

@kdmccormick kdmccormick deleted the lc-course-prototype branch August 14, 2025 17:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@feanil feanil Awaiting requested review from feanil feanil will be requested when the pull request is marked ready for review feanil is a code owner

@farhan farhan Awaiting requested review from farhan farhan will be requested when the pull request is marked ready for review farhan is a code owner

@irtazaakram irtazaakram Awaiting requested review from irtazaakram irtazaakram will be requested when the pull request is marked ready for review irtazaakram is a code owner

@salman2013 salman2013 Awaiting requested review from salman2013 salman2013 will be requested when the pull request is marked ready for review salman2013 is a code owner

Assignees

No one assigned

Labels

create-sandbox A test sandbox will be created for this PR, using the `open-craft/pr-sandbox-automation` tool

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kdmccormick @open-craft-grove @bradenmacdonald @Agrendalath @ormsbee